Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Improvement] Modify networking + Add collator service #111

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

rnovosad
Copy link

Added network_mode: host mode to the services for better interaction. Added collator service into the list.

Added `network_mode: host` mode to the services for better interaction. Added collator service into the list.
@vercel
Copy link

vercel bot commented Sep 16, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/interlay/interbtc-docs/2WEewqGuhkX2Ghb9Jubg79wJnNQw
✅ Preview: https://interbtc-docs-git-fork-rnovosad-patch-1-interlay.vercel.app

@nud3l
Copy link
Member

nud3l commented Sep 17, 2021

On which OS is that host mode required? At least on Linux and mac the current way should work.

On adding the collator to the vault setup, I'd be inclined to not add that at this point since it will take quite some time to sync up the node and will take a lot more space.

@nud3l nud3l added documentation Improvements or additions to documentation question Further information is requested labels Sep 17, 2021
@rnovosad
Copy link
Author

I've tried it on CentOS v7 and it didn't want to get up.
Updated docker-compose file tested both on CentOS 7 and on Mac with BigSur no other steps are needed, all worked fine.
If you want me to test more just tell which OS' I should take and I'll verify.

Regarding collator synchronization, I think it will take less time than bitcoin core.
Anyways, you are the boss, so will do what you say :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants